Skip to content

Fix enable agent re-enabling disabled repos#1126

Merged
peyton-alt merged 1 commit intoentireio:mainfrom
ChetanReddyC:fix-enable-agent-reenable
May 7, 2026
Merged

Fix enable agent re-enabling disabled repos#1126
peyton-alt merged 1 commit intoentireio:mainfrom
ChetanReddyC:fix-enable-agent-reenable

Conversation

@ChetanReddyC
Copy link
Copy Markdown
Contributor

Summary

Fixes #988.

entire enable --agent <name> already set Enabled = true, but it saved through the single-target settings path. When .entire/settings.local.json still contained enabled:false from a previous entire disable, that local override continued to win and the repo remained effectively disabled.

This updates the non-interactive agent setup path to use the same enabled-state save behavior as entire enable, keeping project and existing local settings in sync for the enabled state.

Testing

  • mise run lint
  • mise run test
  • Added regression coverage for the disabled local-settings override case

@ChetanReddyC ChetanReddyC requested a review from a team as a code owner May 6, 2026 05:44
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@peyton-alt
Copy link
Copy Markdown
Contributor

peyton-alt commented May 7, 2026

Hey! Thanks so much for the contribution, the PR looks great.

@peyton-alt peyton-alt merged commit bb4c8fa into entireio:main May 7, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

entire enable --agent X doesn't enable when config already exists

2 participants